-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update redirects-transition.conf #276
Conversation
I have updated the presidential matching funds submission pages from transitions to wagtail.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great to me Kathy. Thank you so much for working on this particular project.
Adding to @kathycarothers' branch off dev. This commit adds several missed redirects from previous rounds of redirects and it includes several new redirects identified in the 7/13 transition inventory meeting.
I just made a commit to Kathy's branch off of dev. Added a few previously missed redirects and new redirects identified in the 7/13 transition inventory meeting. This is PR should also be connected to:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kathycarothers Just one minor change:
redirects-transition.conf
Outdated
rewrite ^/law/cfr/11_cfr.pdf https://www.govinfo.gov/content/pkg/CFR-2010-title11-vol1/pdf/CFR-2010-title11-vol1.pdf redirect; | ||
rewrite ^/law/cfr/2014cfr.pdf https://www.govinfo.gov/content/pkg/CFR-2014-title11-vol1/pdf/CFR-2014-title11-vol1.pdf redirect; | ||
rewrite ^/law/cfr/11cfr2015.pdf https://www.govinfo.gov/content/pkg/CFR-2015-title11-vol1/pdf/CFR-2015-title11-vol1.pdf redirect; | ||
xrewrite ^/law/cfr/ej_citation.shtml https://www.fec.gov/legal-resources/regulations/explanations-and-justifications/explanations-and-justifications-citation-index/ redirect; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kathycarothers Please change from xrewrite
to rewrite
at the beginning of line 437.
I have corrected the error for line 437
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks @kathycarothers!
I have updated the presidential matching funds submission pages from transitions to wagtail. The need to be reviewed and deployed.